program.action - Code Metrics - Inspection of "try clean code" - BitPrepared/d1b0 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c8bf84...8e4dac )

by Stefano

created 2016-10-24 00:43 UTC

program.action A

↳ Parent: Project

Complexity

Conditions	1
Paths	5

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	1	Features	0

Metric	Value
cc	1
c	3
b	1
f	0
nc	5
dl	0
loc	60
rs	9.5555
nop	0

2 Functions

Rating	Name	Duplication	Size	Complexity
A	fs.readFile(ꞌconfig.php.distꞌ)	0	6	2
C	fs.readFile(ꞌconfig.php.distꞌ)	0	50	9

How to fix Long Method

var shell = require('shelljs');
var chalk = require('chalk');
var fs = require('fs');

var enviroment = process.env.npm_package_config_enviroment;

shell.echo('Compile Workspace ('+enviroment+')');

shell.cd('applications/workspace/');

fs.readFile('config.php.dist', 'utf8', function (err,data) {

  if (err) {
    process.stderr.write(chalk.red("Errore in lettura: "+err) + "\n");
    process.exit(1);

  }

  var result;
  if ( enviroment === 'dev' ){
    result = data.replace(/##TYPE##/g, 'sqlite');
    result = result.replace(/##HOST##/g, 'ROOT_PATH.\'/../database/workspace.sqlite\'');
    result = result.replace(/##DBNAME##/g, '');
    result = result.replace(/##PASSWORD##/g, '');
    result = result.replace(/##USERNAME##/g, '');
    result = result.replace(/##LOGLEVEL##/g, '\\Monolog\\Logger::DEBUG');
  } else if ( enviroment === 'vagrant' ){
    result = data.replace(/##TYPE##/g, 'mysql');
    result = result.replace(/##HOST##/g, '\'localhost\'');
    result = result.replace(/##DBNAME##/g, 'workspace');
    result = result.replace(/##PASSWORD##/g, 'workspace');
    result = result.replace(/##USERNAME##/g, 'workspace');
    result = result.replace(/##LOGLEVEL##/g, '\\Monolog\\Logger::DEBUG');
  } else {
    // STAGING / PROD
    result = data.replace(/##TYPE##/g, 'mysql');
    if (
      !process.env.npm_package_config_database_host ||
      !process.env.npm_package_config_database_host ||
      !process.env.npm_package_config_database_dbname ||
      !process.env.npm_package_config_database_password ||
      !process.env.npm_package_config_database_username
    ) {
      process.stderr.write(chalk.red("Errore manca la configurazione in .npmrc") + "\n");
      process.exit(1);

    }
    result = result.replace(/##HOST##/g, '\''+process.env.npm_package_config_database_host+'\'');
    result = result.replace(/##DBNAME##/g, process.env.npm_package_config_database_dbname);
    result = result.replace(/##PASSWORD##/g, process.env.npm_package_config_database_password);
    result = result.replace(/##USERNAME##/g, process.env.npm_package_config_database_username);
    result = result.replace(/##LOGLEVEL##/g, '\\Monolog\\Logger::INFO');
  }


  fs.writeFile('config.php', result, 'utf8', function (err) {
     if (err) {
       process.stderr.write(chalk.red("Errore in scrittura: "+err));
       process.exit(1);

     }
  });
});


1			var shell = require('shelljs');
2			var chalk = require('chalk');
3			var fs = require('fs');
4
5			var enviroment = process.env.npm_package_config_enviroment;
6
7			shell.echo('Compile Workspace ('+enviroment+')');
8
9			shell.cd('applications/workspace/');
10
11			fs.readFile('config.php.dist', 'utf8', function (err,data) {
12
13			if (err) {
14			process.stderr.write(chalk.red("Errore in lettura: "+err) + "\n");
15			process.exit(1);
			0 ignored issues – show Compatibility Debugging Code Best Practice introduced 2016-10-11 21:05 UTC by Report Bug Copy Issue Report Use of `process.exit()` is discouraged as it will potentially stop the complete node.js application. Consider quitting gracefully instead by throwing an Error. Loading history...
16			}
17
18			var result;
19			if ( enviroment === 'dev' ){
20			result = data.replace(/##TYPE##/g, 'sqlite');
21			result = result.replace(/##HOST##/g, 'ROOT_PATH.\'/../database/workspace.sqlite\'');
22			result = result.replace(/##DBNAME##/g, '');
23			result = result.replace(/##PASSWORD##/g, '');
24			result = result.replace(/##USERNAME##/g, '');
25			result = result.replace(/##LOGLEVEL##/g, '\\Monolog\\Logger::DEBUG');
26			} else if ( enviroment === 'vagrant' ){
27			result = data.replace(/##TYPE##/g, 'mysql');
28			result = result.replace(/##HOST##/g, '\'localhost\'');
29			result = result.replace(/##DBNAME##/g, 'workspace');
30			result = result.replace(/##PASSWORD##/g, 'workspace');
31			result = result.replace(/##USERNAME##/g, 'workspace');
32			result = result.replace(/##LOGLEVEL##/g, '\\Monolog\\Logger::DEBUG');
33			} else {
34			// STAGING / PROD
35			result = data.replace(/##TYPE##/g, 'mysql');
36			if (
37			!process.env.npm_package_config_database_host \|\|
38			!process.env.npm_package_config_database_host \|\|
39			!process.env.npm_package_config_database_dbname \|\|
40			!process.env.npm_package_config_database_password \|\|
41			!process.env.npm_package_config_database_username
42			) {
43			process.stderr.write(chalk.red("Errore manca la configurazione in .npmrc") + "\n");
44			process.exit(1);
			0 ignored issues – show Compatibility Debugging Code Best Practice introduced 2016-10-11 21:05 UTC by Report Bug Copy Issue Report Use of `process.exit()` is discouraged as it will potentially stop the complete node.js application. Consider quitting gracefully instead by throwing an Error. Loading history...
45			}
46			result = result.replace(/##HOST##/g, '\''+process.env.npm_package_config_database_host+'\'');
47			result = result.replace(/##DBNAME##/g, process.env.npm_package_config_database_dbname);
48			result = result.replace(/##PASSWORD##/g, process.env.npm_package_config_database_password);
49			result = result.replace(/##USERNAME##/g, process.env.npm_package_config_database_username);
50			result = result.replace(/##LOGLEVEL##/g, '\\Monolog\\Logger::INFO');
51			}
52
53
54			fs.writeFile('config.php', result, 'utf8', function (err) {
55			if (err) {
56			process.stderr.write(chalk.red("Errore in scrittura: "+err));
57			process.exit(1);
			0 ignored issues – show Compatibility Debugging Code Best Practice introduced 2016-10-11 21:05 UTC by Report Bug Copy Issue Report Use of `process.exit()` is discouraged as it will potentially stop the complete node.js application. Consider quitting gracefully instead by throwing an Error. Loading history...
58			}
59			});
60			});
61

BitPrepared / d1b0

GitHub Access Token became invalid

Push — master ( c8bf84...8e4dac )

program.action A

Complexity

Size

Duplication

Importance

2 Functions

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like